Show full dynamic context in SystemPromptEvent.visualize#2273
Merged
Conversation
Remove the 500 character truncation limit for dynamic_context in the visualize method. This helps SDK developers understand if dynamic context is working correctly by showing the complete content. Co-authored-by: openhands <openhands@all-hands.dev>
Contributor
API breakage checks (Griffe)Result: Failed Log excerpt (first 1000 characters) |
Contributor
Agent server REST API breakage checks (OpenAPI)Result: Passed |
all-hands-bot
approved these changes
Mar 3, 2026
Collaborator
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🟢 Good taste
This is exactly the kind of simplification that improves code quality:
- Eliminates special case logic (no more arbitrary 500-char limit)
- Reduces 4 lines to 1 line
- Solves a real problem (debugging difficulty)
The original truncation was arbitrary and made debugging harder. This change is pragmatic—when developers call visualize() for debugging, they want to see the full context.
VERDICT: ✅ LGTM
KEY INSIGHT: Sometimes the best code change is removing code. Arbitrary limits are technical debt.
zparnold
added a commit
to zparnold/software-agent-sdk
that referenced
this pull request
Mar 5, 2026
) Cherry-pick from upstream 9f521a4
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Remove the 500 character truncation limit for
dynamic_contextin theSystemPromptEvent.visualizemethod.Motivation
When debugging or developing with the SDK, it's useful to see the complete dynamic context to verify that everything is working correctly. The previous truncation to 500 characters made it difficult to inspect the full context being passed to the LLM.
Changes
dynamic_contextto 500 characters with "..." suffixdynamic_context.textcontentBefore
After
Agent Server images for this PR
• GHCR package: https://github.com/OpenHands/agent-sdk/pkgs/container/agent-server
Variants & Base Images
eclipse-temurin:17-jdknikolaik/python-nodejs:python3.12-nodejs22golang:1.21-bookwormPull (multi-arch manifest)
# Each variant is a multi-arch manifest supporting both amd64 and arm64 docker pull ghcr.io/openhands/agent-server:dc68dcd-pythonRun
All tags pushed for this build
About Multi-Architecture Support
dc68dcd-python) is a multi-arch manifest supporting both amd64 and arm64dc68dcd-python-amd64) are also available if needed